Merge openldap (2.4.47+dfsg-3+deb10u3) import into refs/heads/workingbranch

[PATCH] ITS#9370 check for equality rule on old_rdn

Just skip normalization if there's no equality rule. We accept
DNs without equality rules already.

Gbp-Pq: Name ITS-9370-check-for-equality-rule-on-old_rdn.patch

[PATCH] ITS#9202 limit depth of nested filters

Using a hardcoded limit for now; no reasonable apps
should ever run into it.

Gbp-Pq: Name ITS-9202-limit-depth-of-nested-filters.patch

[PATCH] ITS#8964 Do not free original filter

Gbp-Pq: Name ITS-8964-Do-not-free-original-filter.patch

[PATCH] ITS#9052 zero out sasl_ssf in connection_init

Gbp-Pq: Name ITS-9052-zero-out-sasl_ssf-in-connection_init.patch

[PATCH] ITS#9038 Another test028 typo

Gbp-Pq: Name ITS-9038-Another-test028-typo.patch

[PATCH] ITS#9038 Fix typo in test script

Gbp-Pq: Name ITS-9038-Fix-typo-in-test-script.patch

[PATCH] ITS#9038 Update test028 to test this is enforced

Gbp-Pq: Name ITS-9038-Update-test028-to-test-this-is-enforced.patch

[PATCH] ITS#9038 restrict rootDN proxyauthz to its own DBs.

Treat as normal user for any other DB.

Gbp-Pq: Name ITS-9038-restrict-rootDN-proxyauthz-to-its-own-DBs.patch

Do not call gnutls_global_set_mutex()

Bug-Debian: https://bugs.debian.org/803197
Forwarded: no

Since GnuTLS moved to implicit initialization on library load, calling
this function deinitializes GnuTLS and then re-initializes it.

When GnuTLS uses /dev/urandom as an entropy source (getrandom() not
available, or older versions of GnuTLS), and the application closed all
file descriptors at startup, this could result in GnuTLS opening
/dev/urandom over one of the application's file descriptors when
re-initialized.

Additionally, the custom mutex functions are never reset, so if libldap
is unloaded (for example via dlclose()) after calling this, its code
may be unmapped and the application could crash when GnuTLS calls the
mutex functions.

The default behaviour of GnuTLS, using pthreads, should be suitable on
all Debian systems, and is probably the same as what libldap uses
anyway.

Gbp-Pq: Name no-gnutls_global_set_mutex

set-maintainer-name

Gbp-Pq: Name set-maintainer-name

ITS6035-olcauthzregex-needs-restart

Gbp-Pq: Name ITS6035-olcauthzregex-needs-restart.patch

don't second-guess BDB ABI

Bug-Debian: http://bugs.debian.org/651333
Forwarded: not-needed

OpenLDAP upstream conservatively assumes that any change to the version
number of libdb can result in an API-breaking change that could impact
the database.  In Debian, we know that such changes require bumping the
library soname and changing the package name, and demand such rigor from
our package maintainers even when upstreams don't deliver; so any such
check in the source code works against the packaging system by forcing
database upgrades when we know none are required.  Disable this check
so we rely on the packaging system to do its job.

Gbp-Pq: Name no-bdb-ABI-second-guessing

Switch to lt_dlopenadvise() so back_perl can be opened with RTLD_GLOBAL. Open all modules with RTLD_GLOBAL, needed so that back_perl can load non-trivial Perl extensions that require symbols from back_perl.so itself.

Bug-Debian: http://bugs.debian.org/327585

Gbp-Pq: Name switch-to-lt_dlopenadvise-to-get-RTLD_GLOBAL-set.diff

don't use AM_INIT_AUTOMAKE macro when we aren't using automake

Calling AM_INIT_AUTOMAKE() in configure.in serves no purpose if we're not
using automake, and it confuses autoreconf.  Use AC_INIT() instead.

Gbp-Pq: Name no-AM_INIT_AUTOMAKE

fix-build-top-mk

Gbp-Pq: Name fix-build-top-mk

add-tlscacert-option-to-ldap-conf

Gbp-Pq: Name add-tlscacert-option-to-ldap-conf

ldap-conf-tls-cacertdir

Gbp-Pq: Name ldap-conf-tls-cacertdir

lastbind-makefile-manpage

Gbp-Pq: Name lastbind-makefile-manpage

[PATCH] contrib/smbk5pwd: add man page, install it too

Add a manual page slapo-smbk5pwd.5 and update smbk5pwd's Makefile to
install the new manual page.

This patch is derived from the corresponding patch upstreamed in ITS#8205

Gbp-Pq: Name smbk5pwd-makefile-manpage

contrib-makefiles

Gbp-Pq: Name contrib-makefiles

do-not-second-guess-sonames

Rip out code that second-guesses the libsasl soname / Debian shlibs.  If
cyrus sasl upstream is breaking the ABI, this needs to be fixed upstream
there, not kludged around upstream here!

Debian bug #546885

Upstream ITS #6302 filed.

Gbp-Pq: Name do-not-second-guess-sonames

getaddrinfo-is-threadsafe

OpenLDAP upstream conservatively assumes that certain resolver functions
(getaddrinfo, getnameinfo, res_query, dn_expand) are not re-entrant; but we
know that the glibc implementations of these functions are thread-safe, so
we should bypass the use of this mutex.  This fixes a locking problem when
an application uses libldap and libnss-ldap is also used for hosts
resolution.

Closes Debian bug #340601.

Not suitable for forwarding upstream; might be made suitable by adding a
configure-time check for glibc and disabling the mutex only on known
thread-safe implementations.

Gbp-Pq: Name getaddrinfo-is-threadsafe

libldap-symbol-versions

Add symbol versioning to the public LDAP libraries.  This is required for
library transitions, such as the current transition from 2.1 to 2.4,
since programs will sometimes have both libraries loaded by different
dependency chains during the transition.

Not yet contributed upstream.

Upstream ITS #5365 filed requesting symbol versioning for libldap and
libber.

Gbp-Pq: Name libldap-symbol-versions

sasl-default-path

Add /etc/ldap/sasl2 to the SASL configuration search path.

Not submitted upstream.  Somewhat Debian-specific and probably not of
interest upstream.

Gbp-Pq: Name sasl-default-path

index-files-created-as-root

Document in the man page that slapindex should be run as the same user
as slapd, and print a warning if it's run as root (since Debian defaults
to running slapd as openldap).

Not suitable for upstream in this form.  This patch needs to be reworked
to check the BerkeleyDB database ownership and only warn if running as
root with a database that's not owned by root.

Upstream ITS #5356 filed requesting better handling of this.  Current
upstream discussion leans towards putting the check into the database
backend and aborting if slapd is run as a different user than the database
owner, which is an even better fix.

Gbp-Pq: Name index-files-created-as-root

wrong-database-location

Move the default slapd database location to /var/lib/ldap instead of
/var/openldap-data.

Debian-specific.

Gbp-Pq: Name wrong-database-location

ldapi-socket-place

Move the ldapi socket to /var/run/slapd from /var/run, since /var/run
is only writable by root and slapd runs as openldap.

Debian-specific.

Gbp-Pq: Name ldapi-socket-place

slapi-errorlog-file

The slapi error log file defaults to /var/errors given our setting
of --localstatedir.  Move it to /var/log/slapi-errors instead.

Debian-specific.

Gbp-Pq: Name slapi-errorlog-file

evolution-ntlm

Patch from evolution-exchange (2.10.3).  The ldap_ntlm_bind function is
actually called by evolution-data-server, checked at version 1.12.2.
Without this patch, the Exchange addressbook integration uses simple binds
with cleartext passwords.

Russ checked with openldap-software for upstream's opinion on this patch
on 2007-12-21.  Upstream had never received it as a patch submission and
given that it's apparently only for older Exchange servers that can't do
SASL and DIGEST-MD5, it's not very appealing.

Bug#457374 filed against evolution-data-server asking if this support is
still required on 2007-12-21.

Gbp-Pq: Name evolution-ntlm

man-slapd

Patch the slapd man page to not refer to a header file that isn't
installed with the slapd package and to reference the correct path
for slapd.

Debian-specific.

Gbp-Pq: Name man-slapd

openldap (2.4.47+dfsg-3+deb10u3) buster-security; urgency=high

  * Fix slapd normalization handling with modrdn (ITS#9370)

[dgit import unpatched openldap 2.4.47+dfsg-3+deb10u3]

Import openldap_2.4.47+dfsg-3+deb10u3.debian.tar.xz

[dgit import tarball openldap 2.4.47+dfsg-3+deb10u3 openldap_2.4.47+dfsg-3+deb10u3.debian.tar.xz]

Merge openldap (2.4.47+dfsg-3+deb10u2) import into refs/heads/workingbranch

[PATCH] ITS#9202 limit depth of nested filters

Using a hardcoded limit for now; no reasonable apps
should ever run into it.

Gbp-Pq: Name ITS-9202-limit-depth-of-nested-filters.patch

[PATCH] ITS#8964 Do not free original filter

Gbp-Pq: Name ITS-8964-Do-not-free-original-filter.patch

[PATCH] ITS#9052 zero out sasl_ssf in connection_init

Gbp-Pq: Name ITS-9052-zero-out-sasl_ssf-in-connection_init.patch

[PATCH] ITS#9038 Another test028 typo

Gbp-Pq: Name ITS-9038-Another-test028-typo.patch

[PATCH] ITS#9038 Fix typo in test script

Gbp-Pq: Name ITS-9038-Fix-typo-in-test-script.patch

[PATCH] ITS#9038 Update test028 to test this is enforced

Gbp-Pq: Name ITS-9038-Update-test028-to-test-this-is-enforced.patch

[PATCH] ITS#9038 restrict rootDN proxyauthz to its own DBs.

Treat as normal user for any other DB.

Gbp-Pq: Name ITS-9038-restrict-rootDN-proxyauthz-to-its-own-DBs.patch

Do not call gnutls_global_set_mutex()

Bug-Debian: https://bugs.debian.org/803197
Forwarded: no

Since GnuTLS moved to implicit initialization on library load, calling
this function deinitializes GnuTLS and then re-initializes it.

When GnuTLS uses /dev/urandom as an entropy source (getrandom() not
available, or older versions of GnuTLS), and the application closed all
file descriptors at startup, this could result in GnuTLS opening
/dev/urandom over one of the application's file descriptors when
re-initialized.

Additionally, the custom mutex functions are never reset, so if libldap
is unloaded (for example via dlclose()) after calling this, its code
may be unmapped and the application could crash when GnuTLS calls the
mutex functions.

The default behaviour of GnuTLS, using pthreads, should be suitable on
all Debian systems, and is probably the same as what libldap uses
anyway.

Gbp-Pq: Name no-gnutls_global_set_mutex

set-maintainer-name

Gbp-Pq: Name set-maintainer-name

ITS6035-olcauthzregex-needs-restart

Gbp-Pq: Name ITS6035-olcauthzregex-needs-restart.patch

don't second-guess BDB ABI

Bug-Debian: http://bugs.debian.org/651333
Forwarded: not-needed

OpenLDAP upstream conservatively assumes that any change to the version
number of libdb can result in an API-breaking change that could impact
the database.  In Debian, we know that such changes require bumping the
library soname and changing the package name, and demand such rigor from
our package maintainers even when upstreams don't deliver; so any such
check in the source code works against the packaging system by forcing
database upgrades when we know none are required.  Disable this check
so we rely on the packaging system to do its job.

Gbp-Pq: Name no-bdb-ABI-second-guessing

Switch to lt_dlopenadvise() so back_perl can be opened with RTLD_GLOBAL. Open all modules with RTLD_GLOBAL, needed so that back_perl can load non-trivial Perl extensions that require symbols from back_perl.so itself.

Bug-Debian: http://bugs.debian.org/327585

Gbp-Pq: Name switch-to-lt_dlopenadvise-to-get-RTLD_GLOBAL-set.diff

don't use AM_INIT_AUTOMAKE macro when we aren't using automake

Calling AM_INIT_AUTOMAKE() in configure.in serves no purpose if we're not
using automake, and it confuses autoreconf.  Use AC_INIT() instead.

Gbp-Pq: Name no-AM_INIT_AUTOMAKE

fix-build-top-mk

Gbp-Pq: Name fix-build-top-mk

add-tlscacert-option-to-ldap-conf

Gbp-Pq: Name add-tlscacert-option-to-ldap-conf

ldap-conf-tls-cacertdir

Gbp-Pq: Name ldap-conf-tls-cacertdir

lastbind-makefile-manpage

Gbp-Pq: Name lastbind-makefile-manpage

[PATCH] contrib/smbk5pwd: add man page, install it too

Add a manual page slapo-smbk5pwd.5 and update smbk5pwd's Makefile to
install the new manual page.

This patch is derived from the corresponding patch upstreamed in ITS#8205

Gbp-Pq: Name smbk5pwd-makefile-manpage

contrib-makefiles

Gbp-Pq: Name contrib-makefiles

do-not-second-guess-sonames

Rip out code that second-guesses the libsasl soname / Debian shlibs.  If
cyrus sasl upstream is breaking the ABI, this needs to be fixed upstream
there, not kludged around upstream here!

Debian bug #546885

Upstream ITS #6302 filed.

Gbp-Pq: Name do-not-second-guess-sonames

getaddrinfo-is-threadsafe

OpenLDAP upstream conservatively assumes that certain resolver functions
(getaddrinfo, getnameinfo, res_query, dn_expand) are not re-entrant; but we
know that the glibc implementations of these functions are thread-safe, so
we should bypass the use of this mutex.  This fixes a locking problem when
an application uses libldap and libnss-ldap is also used for hosts
resolution.

Closes Debian bug #340601.

Not suitable for forwarding upstream; might be made suitable by adding a
configure-time check for glibc and disabling the mutex only on known
thread-safe implementations.

Gbp-Pq: Name getaddrinfo-is-threadsafe

libldap-symbol-versions

Add symbol versioning to the public LDAP libraries.  This is required for
library transitions, such as the current transition from 2.1 to 2.4,
since programs will sometimes have both libraries loaded by different
dependency chains during the transition.

Not yet contributed upstream.

Upstream ITS #5365 filed requesting symbol versioning for libldap and
libber.

Gbp-Pq: Name libldap-symbol-versions

sasl-default-path

Add /etc/ldap/sasl2 to the SASL configuration search path.

Not submitted upstream.  Somewhat Debian-specific and probably not of
interest upstream.

Gbp-Pq: Name sasl-default-path

index-files-created-as-root

Document in the man page that slapindex should be run as the same user
as slapd, and print a warning if it's run as root (since Debian defaults
to running slapd as openldap).

Not suitable for upstream in this form.  This patch needs to be reworked
to check the BerkeleyDB database ownership and only warn if running as
root with a database that's not owned by root.

Upstream ITS #5356 filed requesting better handling of this.  Current
upstream discussion leans towards putting the check into the database
backend and aborting if slapd is run as a different user than the database
owner, which is an even better fix.

Gbp-Pq: Name index-files-created-as-root

wrong-database-location

Move the default slapd database location to /var/lib/ldap instead of
/var/openldap-data.

Debian-specific.

Gbp-Pq: Name wrong-database-location

ldapi-socket-place

Move the ldapi socket to /var/run/slapd from /var/run, since /var/run
is only writable by root and slapd runs as openldap.

Debian-specific.

Gbp-Pq: Name ldapi-socket-place

slapi-errorlog-file

The slapi error log file defaults to /var/errors given our setting
of --localstatedir.  Move it to /var/log/slapi-errors instead.

Debian-specific.

Gbp-Pq: Name slapi-errorlog-file

evolution-ntlm

Patch from evolution-exchange (2.10.3).  The ldap_ntlm_bind function is
actually called by evolution-data-server, checked at version 1.12.2.
Without this patch, the Exchange addressbook integration uses simple binds
with cleartext passwords.

Russ checked with openldap-software for upstream's opinion on this patch
on 2007-12-21.  Upstream had never received it as a patch submission and
given that it's apparently only for older Exchange servers that can't do
SASL and DIGEST-MD5, it's not very appealing.

Bug#457374 filed against evolution-data-server asking if this support is
still required on 2007-12-21.

Gbp-Pq: Name evolution-ntlm

man-slapd

Patch the slapd man page to not refer to a header file that isn't
installed with the slapd package and to reference the correct path
for slapd.

Debian-specific.

Gbp-Pq: Name man-slapd

openldap (2.4.47+dfsg-3+deb10u2) buster-security; urgency=high

  * Fix slapd to limit depth of nested expressions in search filters
    (ITS#9202)

[dgit import unpatched openldap 2.4.47+dfsg-3+deb10u2]

Import openldap_2.4.47+dfsg-3+deb10u2.debian.tar.xz

[dgit import tarball openldap 2.4.47+dfsg-3+deb10u2 openldap_2.4.47+dfsg-3+deb10u2.debian.tar.xz]

Merge openldap (2.4.47+dfsg-3+deb10u1) import into refs/heads/workingbranch

ITS#8964 Do not free original filter

Gbp-Pq: Name ITS-8964-Do-not-free-original-filter.patch

ITS#9052 zero out sasl_ssf in connection_init

Gbp-Pq: Name ITS-9052-zero-out-sasl_ssf-in-connection_init.patch

ITS#9038 Another test028 typo

Gbp-Pq: Name ITS-9038-Another-test028-typo.patch

ITS#9038 Fix typo in test script

Gbp-Pq: Name ITS-9038-Fix-typo-in-test-script.patch

ITS#9038 Update test028 to test this is enforced

Gbp-Pq: Name ITS-9038-Update-test028-to-test-this-is-enforced.patch

ITS#9038 restrict rootDN proxyauthz to its own DBs.

Treat as normal user for any other DB.

Gbp-Pq: Name ITS-9038-restrict-rootDN-proxyauthz-to-its-own-DBs.patch

no-gnutls_global_set_mutex

Gbp-Pq: Name no-gnutls_global_set_mutex

set-maintainer-name

Gbp-Pq: Name set-maintainer-name

ITS6035-olcauthzregex-needs-restart

Gbp-Pq: Name ITS6035-olcauthzregex-needs-restart.patch

no-bdb-ABI-second-guessing

Gbp-Pq: Name no-bdb-ABI-second-guessing

Switch to lt_dlopenadvise() so back_perl can be opened with RTLD_GLOBAL. Open all modules with RTLD_GLOBAL, needed so that back_perl can load non-trivial Perl extensions that require symbols from back_perl.so itself.

Gbp-Pq: Name switch-to-lt_dlopenadvise-to-get-RTLD_GLOBAL-set.diff

no-AM_INIT_AUTOMAKE

Gbp-Pq: Name no-AM_INIT_AUTOMAKE

fix-build-top-mk

Gbp-Pq: Name fix-build-top-mk

add-tlscacert-option-to-ldap-conf

Gbp-Pq: Name add-tlscacert-option-to-ldap-conf

ldap-conf-tls-cacertdir

Gbp-Pq: Name ldap-conf-tls-cacertdir

lastbind-makefile-manpage

Gbp-Pq: Name lastbind-makefile-manpage

contrib/smbk5pwd: add man page, install it too

Add a manual page slapo-smbk5pwd.5 and update smbk5pwd's Makefile to
install the new manual page.

This patch is derived from the corresponding patch upstreamed in ITS#8205

Gbp-Pq: Name smbk5pwd-makefile-manpage

contrib-makefiles

Gbp-Pq: Name contrib-makefiles

do-not-second-guess-sonames

Rip out code that second-guesses the libsasl soname / Debian shlibs.  If
cyrus sasl upstream is breaking the ABI, this needs to be fixed upstream
there, not kludged around upstream here!

Debian bug #546885

Upstream ITS #6302 filed.

Gbp-Pq: Name do-not-second-guess-sonames

getaddrinfo-is-threadsafe

OpenLDAP upstream conservatively assumes that certain resolver functions
(getaddrinfo, getnameinfo, res_query, dn_expand) are not re-entrant; but we
know that the glibc implementations of these functions are thread-safe, so
we should bypass the use of this mutex.  This fixes a locking problem when
an application uses libldap and libnss-ldap is also used for hosts
resolution.

Closes Debian bug #340601.

Not suitable for forwarding upstream; might be made suitable by adding a
configure-time check for glibc and disabling the mutex only on known
thread-safe implementations.

Gbp-Pq: Name getaddrinfo-is-threadsafe

libldap-symbol-versions

Add symbol versioning to the public LDAP libraries.  This is required for
library transitions, such as the current transition from 2.1 to 2.4,
since programs will sometimes have both libraries loaded by different
dependency chains during the transition.

Not yet contributed upstream.

Upstream ITS #5365 filed requesting symbol versioning for libldap and
libber.

Gbp-Pq: Name libldap-symbol-versions

sasl-default-path

Add /etc/ldap/sasl2 to the SASL configuration search path.

Not submitted upstream.  Somewhat Debian-specific and probably not of
interest upstream.

Gbp-Pq: Name sasl-default-path

index-files-created-as-root

Document in the man page that slapindex should be run as the same user
as slapd, and print a warning if it's run as root (since Debian defaults
to running slapd as openldap).

Not suitable for upstream in this form.  This patch needs to be reworked
to check the BerkeleyDB database ownership and only warn if running as
root with a database that's not owned by root.

Upstream ITS #5356 filed requesting better handling of this.  Current
upstream discussion leans towards putting the check into the database
backend and aborting if slapd is run as a different user than the database
owner, which is an even better fix.

Gbp-Pq: Name index-files-created-as-root

wrong-database-location

Move the default slapd database location to /var/lib/ldap instead of
/var/openldap-data.

Debian-specific.

Gbp-Pq: Name wrong-database-location

ldapi-socket-place

Move the ldapi socket to /var/run/slapd from /var/run, since /var/run
is only writable by root and slapd runs as openldap.

Debian-specific.

Gbp-Pq: Name ldapi-socket-place

slapi-errorlog-file

The slapi error log file defaults to /var/errors given our setting
of --localstatedir.  Move it to /var/log/slapi-errors instead.

Debian-specific.

Gbp-Pq: Name slapi-errorlog-file

evolution-ntlm

Patch from evolution-exchange (2.10.3).  The ldap_ntlm_bind function is
actually called by evolution-data-server, checked at version 1.12.2.
Without this patch, the Exchange addressbook integration uses simple binds
with cleartext passwords.

Russ checked with openldap-software for upstream's opinion on this patch
on 2007-12-21.  Upstream had never received it as a patch submission and
given that it's apparently only for older Exchange servers that can't do
SASL and DIGEST-MD5, it's not very appealing.

Bug#457374 filed against evolution-data-server asking if this support is
still required on 2007-12-21.

Gbp-Pq: Name evolution-ntlm

man-slapd

Patch the slapd man page to not refer to a header file that isn't
installed with the slapd package and to reference the correct path
for slapd.

Debian-specific.

Gbp-Pq: Name man-slapd

openldap (2.4.47+dfsg-3+deb10u1) buster; urgency=medium

  * Fix slapd to restrict rootDN proxyauthz to its own databases
    (CVE-2019-13057) (ITS#9038) (Closes: #932997)
  * Fix slapd to enforce sasl_ssf ACL statement on every connection
    (CVE-2019-13565) (ITS#9052) (Closes: #932998)
  * Fix slapo-rwm to not free original filter when rewritten filter is invalid
    (ITS#8964) (Closes: #934277, LP: #1838370)

[dgit import unpatched openldap 2.4.47+dfsg-3+deb10u1]

Import openldap_2.4.47+dfsg-3+deb10u1.debian.tar.xz

[dgit import tarball openldap 2.4.47+dfsg-3+deb10u1 openldap_2.4.47+dfsg-3+deb10u1.debian.tar.xz]

Merge openldap (2.4.47+dfsg-3) import into refs/heads/workingbranch

no-gnutls_global_set_mutex

Gbp-Pq: Name no-gnutls_global_set_mutex

set-maintainer-name

Gbp-Pq: Name set-maintainer-name

ITS6035-olcauthzregex-needs-restart

Gbp-Pq: Name ITS6035-olcauthzregex-needs-restart.patch